-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[stable23] Update autoloaders and Node package-lock.json #31862
Conversation
Strange,
Looks like those are in Ah, now I see a warning, looks like I need to downgrade Node and npm:
EDIT: Doesn't change anything. Strange is also that I do not see this error when running These errors btw showed up the first time with this commit: b404e3f Also I do not understand the error, the two react modules are actual dependencies, e.g.: "node_modules/linkifyjs": {
"version": "2.1.9",
"resolved": "https://registry.npmjs.org/linkifyjs/-/linkifyjs-2.1.9.tgz",
"integrity": "sha512-74ivurkK6WHvHFozVaGtQWV38FzBwSTGNmJolEgFp7QgR2bl6ArUWlvT4GcHKbPe1z3nWYi+VUdDZk16zDOVug==",
"peerDependencies": {
"jquery": ">= 1.11.0",
"react": ">= 0.14.0",
"react-dom": ">= 0.14.0"
}
}, And Not idea what is wrong with the CI node checks, I cannot replicate anything locally, help needed. |
I guess we need some help here. Maybe @artonge has some ideas how to solve this? :) |
The issue might come from npm |
Ah, so the local build was not the issue, but since the workflows installs
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please rerun the npm step with npm ci
for CI
29e8c20
to
05f8768
Compare
Signed-off-by: MichaIng <micha@dietpi.com>
05f8768
to
81af11f
Compare
Okay worked now. The three Node modules have now been added to Dependency tree seems to be:
|
Honestly can't tell. If linkifyjs requires peerDeps, so be it 🤷 |
Autoloaders update is required to solve Drone checkers,
package-lock.json
update to solve all Node checks.I simply did:
PHP version was 8.1, in case it is relevant.